feat: add accessibility features for pie menus and toolbar speech syn…#5592
feat: add accessibility features for pie menus and toolbar speech syn…#5592Siddharth-732 wants to merge 6 commits intosugarlabs:masterfrom
Conversation
|
✅ All Jest tests passed! This PR is ready to merge. |
|
✅ All Jest tests passed! This PR is ready to merge. |
|
✅ All Jest tests passed! This PR is ready to merge. |
|
❌ Some Jest tests failed. Please check the logs and fix the issues before merging. Failed Tests: |
|
@walterbender @omsuneri Please review the changes if they are helpful for the platform. Thank you |
|
@Siddharth-732 See Jest cases are failing. |
|
❌ Some Jest tests failed. Please check the logs and fix the issues before merging. Failed Tests: |
|
✅ All Jest tests passed! This PR is ready to merge. |
|
Jest Test is passing now, please check if it is ready to be merged |
|
This is really interesting. I am not sure about the placement of the control. Maybe it belongs on the main toolbar? @pikurasa what do you think? |
|
@walterbender I'm glad you liked the feature, let me know about any changes you would like to have in my solution, I'll be more than happy to implement them. |
fixes #5591
Summary
Add optional text-to-speech (TTS) feedback for pie-menu selections and toolbar Play/Stop actions, plus a top-right toggle to enable/disable speech. This improves accessibility for users who rely on audio confirmation.
Approach
aria-liveregion.speechSynthesisto speak labels on selection when the global toggle is active.Affected Files
piemenus.js— Label normalization,aria-liveannouncements, TTS hooks, global toggle flag.wheelnav.js— Triggers for announcements and TTS on menu item clicks.artwork.js— Added new speaker icon SVG for the toggle button.turtles.js— Toggle button implementation and top-right layout adjustments.toolbar.js— Implementation for Play/Stop button speech logic.Video Proof
Demo.mp4
Notes for Maintainers
Environment